Conversation
…nal method names - Remove unused find_flowllm import and export from utils module - Rename parse_env_file to _parse_env_file to indicate internal usage - Rename get_version to _get_version in logo_utils to indicate internal usage - Rename locate_flowllm to _locate_flowllm in service_utils to indicate internal usage - Update all references to use new private method names - Clean up formatting in various function definitions by removing unnecessary indentation - Fix exponentiation operator syntax from 2 ** attempt to 2**attempt - Adjust slice notation from i: i + size to i : i + size for consistency - Remove deprecated traverse step tests from common steps unit tests - Remove unused cosine_similarity import from embedding integration tests - Remove unused default config test for daily write job registration - Add new CLI unit test for call_server client kwargs passing
…n clarity - Consolidated imports and removed unnecessary blank lines across multiple files - Simplified docstrings to be more concise while preserving essential information - Removed unused variables and redundant comments throughout the codebase - Updated WorkspaceEnv to look for session-state files in session/ instead of resource/ - Modified AsAgentWrapper to remove verbose logging and streamline session handling - Adjusted BackgroundJob docstring to be more focused on core functionality - Removed unused import and simplified method signatures in various components - Refactored Application class to reduce directory creation and simplify component initialization - Updated test fixture documentation to be more direct and clear
- Removed auto-dream-and-proactive.svg flow chart - Removed auto-index-and-memory-search.svg flow chart - Removed auto-memory-resource.svg flow chart - Removed old auto_cognition_design.md document - Cleaned up deprecated visualization assets - Removed redundant design documentation files
- Create contributing guidelines with development conventions and commit standards - Add detailed framework documentation covering directory structure and architecture - Include quick start guide with installation and basic usage instructions - Add architecture diagrams for visual representation of FlowLLM framework - Document service registration, job execution, and component lifecycle patterns
- Condensed core features description for better readability - Removed detailed technical explanations from feature list - Simplified open source and contributing section - Removed conventional commits examples from README - Added star history chart to both English and Chinese README - Updated license description to be more concise
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Background
This PR is a large framework-level refactor, so I updated the PR title to follow the repository contribution guideline's Conventional Commits format:
refactor(framework): introduce component-based application architectureThe change moves FlowLLM away from the older
core/op/gallery/extensionslayout and consolidates the runtime around the documented main path:Target behavior
The intended direction appears to be a cleaner, configuration-driven application framework:
flowllm.application.Applicationbecomes the main entry point for wiring services, components, and jobs.flowllm/components/and is built aroundBaseComponent.flowllm/steps/and can be composed into Jobs.flowllm/config/default.yaml, keeping the defaultflowllm startpath discoverable.flowllm.schema,flowllm.enumeration,flowllm.config, andflowllm.utils.Impact scope
This is best reviewed as a major architectural reset rather than a narrow refactor.
The PR:
${ENV_VAR}/${ENV_VAR:-default}.tests/unitandtests/integration.flowllm.core,flowllm.extensions,flowllm.gallery, legacy vector store code, older docs, and theold/tree.Because of the package removals, this should be treated as a breaking change for users importing old modules directly.
Test plan
The PR adds or updates coverage for:
BaseComponentlifecycle and dependency injection behavior.Before merge, the expected validation should match the contribution guide:
For focused checks:
One CI issue should be fixed or confirmed:
.github/workflows/unittest.ymlcurrently installspip install -e ".[dev,core]", butpyproject.tomldefinesdev,claude-code, andfullextras, notcore. If thecoreextra was intentionally removed, CI should probably use.[dev]or.[full].Review focus
I recommend reviewers focus on these boundaries:
flowllm.__init__,flowllm.application,flowllm.components,flowllm.schema, andflowllm.stepsare the intended stable API.flowllm.core,flowllm.extensions, andflowllm.galleryis acceptable for this release.flowllm/config/default.yamlis registered through@R.register(...)and imported by the relevant__init__.py.flowllm = flowllm.application:maincovers the intended quick-start path.